@microsoft-github-policy-service agree

@iamjustsam : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

Learn Build status updates of commit 8eeb8f5:

✅ Validation status: passed

For more details, please refer to the build report.

For any questions, please:

• Try searching the learn.microsoft.com contributor guides
• Post your question in the Learn support channel

@rwestMSFT - Can you review the proposed changes?

IMPORTANT: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Hi @Pietervanhove - Can you please review to see if this is an appropriate addition the doc? Thanks!

#assign:Pietervanhove

It might be a good idea to add this, but I'm not sure if the error message "500 InternalServerError - An unexpected error occured while processing the request." is only related to a wrong key format. I'll need to double check with engineering.

It might be a good idea to add this, but I'm not sure if the error message "500 InternalServerError - An unexpected error occured while processing the request." is only related to a wrong key format. I'll need to double check with engineering.

@Pietervanhove - Would you add a comment to indicate what action to take on this PR?
Thanks!

It might be a good idea to add this, but I'm not sure if the error message "500 InternalServerError - An unexpected error occured while processing the request." is only related to a wrong key format. I'll need to double check with engineering.

@Pietervanhove - Would you add a comment to indicate what action to take on this PR? Thanks!

I'm waiting for engineering on their feedback. Once I received their feedback, I can comment on what action we should take

Adds a section on unsupported keys. Using an unsupported key will throw an InternalServerError without further information.

Hi,

We have verified our code and as expected we do have an error message that captures the wrong key size. Can you please let me know what action you did and how we can reproduce this?

Thanks,
Pieter

Hi Pieter

You can reproduce this in either of the following ways:

• Deploy a key to an existing SQL Managed Instance using an ARM resource, referencing a key with an unsupported length

  {
    "type": "Microsoft.Sql/managedInstances/keys",
    "apiVersion": "2024-05-01-preview",
    "name": "[concat(parameters('name'), '/', variables('encyptionKeyName'))]",
    "properties": {
      "serverKeyType": "AzureKeyVault",
      "uri": "[parameters('cmkKeyId')]"
    }
  }

  The ARM deployment will fail with the following error:

  {
    "code": "InternalServerError",
    "message": "An unexpected error occured while processing the request. Tracking ID: '...'"
  }

• Deploy a key to an existing SQL Managed Instance using Azure CLI, referencing a key with an unsupported length
  az sql mi key create -kid <KeyId> --managed-instance <ManagedInstance> --resource-group <ManagedInstanceResourceGroup>
  The command will fail with the following output:

  (InternalServerError) An unexpected error occured while processing the request. Tracking ID: '...'
  Code: InternalServerError
  
  Message: An unexpected error occured while processing the request. Tracking ID: '...'
  

Hi Sam,

We are aware of that issue on the MI side - we do not handle this error gracefully in Management Service, so it only appears as an internal server error. We created a repair item to address it. So, for now, I wouldn't add this to the documentation because the internal server error is not unique for the key size issue.

Regards,
Pieter

#please-close

Hi Sam,

We are aware of that issue on the MI side - we do not handle this error gracefully in Management Service, so it only appears as an internal server error. We created a repair item to address it. So, for now, I wouldn't add this to the documentation because the internal server error is not unique for the key size issue.

Regards, Pieter